Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a thesis to citations #1399

Merged
merged 2 commits into from
Apr 26, 2022
Merged

Add a thesis to citations #1399

merged 2 commits into from
Apr 26, 2022

Conversation

marcharper
Copy link
Member

No description provided.

@marcharper
Copy link
Member Author

Error appears unrelated (black wants to reformat some files).

@Nikoleta-v3
Copy link
Member

For some citations we use brackets for the publication year and for some we don't 🤷🏻 Do we care about this? Do we want to be consistent?

@drvinceknight
Copy link
Member

Error appears unrelated (black wants to reformat some files).

This could be due to an update of black.

drvinceknight added a commit that referenced this pull request Mar 1, 2022
This is a PR to #1399 just
to check if running an updated version of black (22.1.0) fixes the CI.
@drvinceknight drvinceknight mentioned this pull request Mar 1, 2022
@drvinceknight
Copy link
Member

Have opened #1400 which is a PR to this branch with an updated version of black. (Do we want to pin black?)

This is a PR to #1399 just
to check if running an updated version of black (22.1.0) fixes the CI.
@drvinceknight
Copy link
Member

This looks like hypothesis has stumbled on an edge case?

@drvinceknight
Copy link
Member

I believe this is good to go.

@meatballs meatballs merged commit 11373a9 into dev Apr 26, 2022
@marcharper marcharper deleted the marcharper-patch-3 branch July 3, 2022 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants